-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace TSVB timeseries charts with elastic-charts #33558
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sulemanof
force-pushed
the
charts/test-lines-series
branch
from
March 22, 2019 11:27
ed102dd
to
6ae639c
Compare
sulemanof
force-pushed
the
charts/test-lines-series
branch
2 times, most recently
from
March 29, 2019 09:58
5d48034
to
7904847
Compare
sulemanof
changed the title
[WIP] Test charts - line series
[WIP] Replace TSVB timeseries charts with elastic-charts
Mar 31, 2019
sulemanof
force-pushed
the
charts/test-lines-series
branch
from
April 1, 2019 09:30
7904847
to
891200b
Compare
2 tasks
💔 Build Failed |
retest |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
3 tasks
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
PhilippBaranovskiy
previously requested changes
Apr 15, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @sulemanof, I guess you might leave conflict marks in the code
💔 Build Failed |
gospodarsky
force-pushed
the
charts/test-lines-series
branch
from
April 15, 2019 13:26
829404c
to
197491f
Compare
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
alexwizp
added
v7.5.0
v8.0.0
Team:Visualizations
Visualization editors, elastic-charts and infrastructure
labels
Sep 3, 2019
Pinging @elastic/kibana-app |
…-series # Conflicts: # test/functional/apps/visualize/_tsvb_time_series.ts
💚 Build Succeeded |
nickofthyme
dismissed
markov00’s stale review
September 4, 2019 16:56
requested change made in 6c17fb8
alexwizp
approved these changes
Sep 5, 2019
💚 Build Succeeded |
💚 Build Succeeded |
💚 Build Succeeded |
alexwizp
pushed a commit
to alexwizp/kibana
that referenced
this pull request
Sep 6, 2019
* Replace TSVB timeseries charts with elastic-charts * Add sort index for series * Update src/legacy/core_plugins/metrics/public/visualizations/views/timeseries/index.js Co-Authored-By: Nick Partridge <nick.ryan.partridge@gmail.com> * Fix PR comments * fix issue with scaling * fix crosshair styles for bar
jloleysens
added a commit
to jloleysens/kibana
that referenced
this pull request
Sep 6, 2019
…ete-for-distance_feature * 'master' of github.com:elastic/kibana: (89 commits) Replace TSVB timeseries charts with elastic-charts (elastic#33558) [TSVB][Top N aggregation] Unable to deal with negative values (elastic#43581) [alerting] Adds Action Type configuration support and whitelisting (elastic#44483) FTR: fix WebDriver Actions calls (elastic#44605) [Code] add NodeRepositoriesService to watch new repositories on local node (elastic#44677) [skip-ci][Maps] Improve Maps intro page (elastic#44721) [Maps] Update titles and descriptions for data sources (elastic#44833) Types + Extract Integration Util (elastic#44433) Downgrade log level from info to debug for cases when we cannot handle authentication attempt. (elastic#44933) [Reporting] Remove Chome stdout/stderr observables, Add Browser Logger observable (elastic#44359) Update Jest script to output coverage (elastic#44447) [ftr] support --kibana-install-dir flag (elastic#44552) [WATCHER] Allow user to set a threshold value of 0 (elastic#44810) Remove injectI18n in dashboard plugin. (elastic#44580) [Graph] Save modal (elastic#44261) Use external script for the OIDC Implicit flow handler page. (elastic#44866) disable router prefixing with pluginId (elastic#44855) [SIEM] Fix bug on url + inspect functionality on hosts/hostDetails page (elastic#44671) [ML] File data viz limiting uploaded doc chunk size (elastic#44768) [code] Append go env variable 'GOCACHE' to go lsp spawn command. (elastic#44864) ...
alexwizp
added a commit
that referenced
this pull request
Sep 6, 2019
* Replace TSVB timeseries charts with elastic-charts * Add sort index for series * Update src/legacy/core_plugins/metrics/public/visualizations/views/timeseries/index.js Co-Authored-By: Nick Partridge <nick.ryan.partridge@gmail.com> * Fix PR comments * fix issue with scaling * fix crosshair styles for bar
jloleysens
added a commit
to jloleysens/kibana
that referenced
this pull request
Sep 6, 2019
…plate * 'master' of github.com:elastic/kibana: (91 commits) [APM] Make number of x ticks responsive to the plot width (elastic#44870) [ML] Single metric viewer: Fix top nav refresh behaviour. (elastic#44860) Replace TSVB timeseries charts with elastic-charts (elastic#33558) [TSVB][Top N aggregation] Unable to deal with negative values (elastic#43581) [alerting] Adds Action Type configuration support and whitelisting (elastic#44483) FTR: fix WebDriver Actions calls (elastic#44605) [Code] add NodeRepositoriesService to watch new repositories on local node (elastic#44677) [skip-ci][Maps] Improve Maps intro page (elastic#44721) [Maps] Update titles and descriptions for data sources (elastic#44833) Types + Extract Integration Util (elastic#44433) Downgrade log level from info to debug for cases when we cannot handle authentication attempt. (elastic#44933) [Reporting] Remove Chome stdout/stderr observables, Add Browser Logger observable (elastic#44359) Update Jest script to output coverage (elastic#44447) [ftr] support --kibana-install-dir flag (elastic#44552) [WATCHER] Allow user to set a threshold value of 0 (elastic#44810) Remove injectI18n in dashboard plugin. (elastic#44580) [Graph] Save modal (elastic#44261) Use external script for the OIDC Implicit flow handler page. (elastic#44866) disable router prefixing with pluginId (elastic#44855) [SIEM] Fix bug on url + inspect functionality on hosts/hostDetails page (elastic#44671) ...
10 tasks
22 tasks
1 task
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Feature:TSVB
TSVB (Time Series Visual Builder)
release_note:enhancement
Team:Visualizations
Visualization editors, elastic-charts and infrastructure
v7.5.0
v8.0.0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolve elastic/elastic-charts#108 .
Should be checked before merging:
In progress replacements:
Blocker Issues:
/Cannot set the custom formatting function for Tooltip header / Annotation header elastic-charts#197/Fill color and Line width styles are not applied correctly to Bars elastic-charts#323/'tickFormat' property should affect only Axis for which it be declared elastic-charts#185/Disable tooltip on annotation line elastic-charts#324/Dynamic legend content size elastic-charts#311/The value on the legend is not displayed completely elastic-charts#184/The Legend is not displayed correctly in the 'Bottom' mode elastic-charts#194/elasticChartsCrosshair is not positioned correctly for some intervals elastic-charts#195/Cannot set line width (strokeWidth) for BarSeries elastic-charts#196/Color Picker functionality cannot be disabled elastic-charts#198/Percent stacked option elastic-charts#222Tests sample dashboards⚠️
Tasks
Issues 🐛
[eCommerce] Promotion Tracking
,[Flights] Delays & Cancellations
, also on visualization editor, the visualization doesn't show the crosshair line (usually red)[Flights] Delays & Cancellations
disable tooltip on annotation line (/Disable tooltip on annotation line elastic-charts#324)[Logs] Response Codes Over Time + Annotations
the order of the series is inverted, meaning that you have to reorder the way you are adding the series into the Chart component. It's reproducible when using stacked mode on a single visualization with a group by split@AlonaNadler please have a look.
this PR
on 7.3
[Logs] Response Codes Over Time + Annotations
the annotation header seems always showing the same timestamp. Also reproducible during tests on Annotations[Logs] Response Codes Over Time + Annotations
I think you are not checking the formatter fornull
values as in a case you can see theNaN
text hereTest visualization editor⚠️
Tasks
Build a single series visualization
On
Panel options
kibana_sample_data_flights
and choosetimestamp
as TimestampDrop last bucket
shows and hide the last bucket on the seriesOn
Data -> Options
tab of a single series:Chart type
line, barChart type
styles: fill, line width, point size, stepsoffset series
,hide in legend
,split color theme
(only when multi series),separate axis
,override index pattern
usingkibana_sample_data_logs
On
Data -> Metrics
tab of a single series and with multiple series:Average
aggregation onAvgTicketPrice
Group by: Terms
by: Origin
Data -> options
optionsOn
Annotations
tabkibana_sample_data_flights
index pattern ,useDest : "Verona Villafranca Airport" and Cancelled :true
as KQL query, select an Icon, useDest
as Fields and{{Dest}}
as row template.Issues 🐛
[x] The point size works fine until the value of 10, after that, 11 or more there is no more fill:this is not an issue, but a feature existing on 7.3 also :(
point size: 10
point size: 11 or above
Fill
andLine width
styles are not applied to BarsHide in legend
seems not working with a single seriesSplit within series
doesn't seem to work.Exclamation in circle
icon Exclamation mark in circle icon eui#2121Test PDF rendering ✅
Enable trial mode
Start a 30-day trial
from Settings -> Licence ManagerNew issues fixed by this PR 🎉
before:
now:
0
value:on PR
on master/7.3
when using stacked in percentage mode, it automatically apply the
%
sign to the data formatwhen using a different timezone than the one in the browser, on the previous version each ticks where rounded to the UTC timezone, losing the nice rounding of the hours/days 00:00, 12:00, 00:00, 12:00
This PR fix that keeping the same rouding
Notes for testing
Compere the visualizations against
master
branch or at least against 7.3.Using docker with 7.3 is a bit more easy: these are the steps for that:
Write a
docker-compose.yaml
file and run it withdocker-compose up
:To use exactly the same dataset (that will allows you to compare also the visualization output use the following from kibana dev tools) this will reindex all the data from the testing elasticsearch into the dockerized ES
Notes for Elastic-Charts 🐛
on 7.3
**on this PR**